Skip to content

Fix/apidocs breadcrumb toc asf#1232

Merged
paulirwin merged 11 commits into
apache:masterfrom
zka26:fix/apidocs-breadcrumb-toc-asf
Apr 10, 2026
Merged

Fix/apidocs breadcrumb toc asf#1232
paulirwin merged 11 commits into
apache:masterfrom
zka26:fix/apidocs-breadcrumb-toc-asf

Conversation

@zka26

@zka26 zka26 commented Jan 11, 2026

Copy link
Copy Markdown
Contributor
  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a change, please open an issue to discuss the change or find an existing issue.

Render apidocs navbar links via DocFX metadata; fix breadcrumb; fix event link

Fixes #1215 (Continuation of #1198)

Description

  • Fixed breadcrumb rendering: the breadcrumb is no longer split across multiple lines and is displayed as a single line with / separators.
  • Updated the ASF “Events” link: adjusted it so it matches the regex.
  • Improved navbar rendering for site + apidocs: ASF menu is emitted as static HTML during DocFX build on both the main site and apidocs.
  • Now the ASF dropdown menu appears in the api / {subsites}.
Képernyőkép 2026-01-11 221609

Introduces _navbarLinks and _asfLinks configuration in docfx.json including the correct event page link. Adds a new navbar-links partial to render these links and updates the main navbar partial to include it.
Now the Api / Libraries works as intended, not splitting into two lines and the slash is restored.
Introduces _navbarLinks and _asfLinks configuration in docfx.global.json including the correct event page link. Adds a new navbar-links partial to render these links and updates the main navbar partial to include it.
- Added docfx.links.json to organize navigation and ASF links separately. It now includes the links.
- Modified docfx.site.json to include docfx.links.json in globalMetadataFiles.
Added docfx.links.json to metadata and updated navbar links to point to the latest CI documentation. Enhanced docs.ps1 to automatically update both docfx.links.json with the current LuceneNetVersion during the build process, ensuring navigation and API links always reference the correct version.
…earing inside the apidocs.

- Refactor navbar links to use individual properties and moved to the subsite.json.
- docfx.links.json and navbar-links.tmpl.partial removed.
- docs.ps1 restored as it was originally.
- Removed the link json file reference from the metadata docfx.core.json.
- Change _navCliHref to absolute URL for published CLI docs
- Removed the links jsons globalMetadataFiles leftover and the subsite one was added instead.
- Auto-update CLI doc version in docs.ps1
- Improve navbar template indentation for readability and removed a debug message from it.
@zka26 zka26 marked this pull request as draft January 11, 2026 22:06
@zka26

zka26 commented Jan 11, 2026

Copy link
Copy Markdown
Contributor Author

Hey @NightOwl888 , @paulirwin!

First of all: Sorry this took so long. Some personal stuff came up and I should have communicated earlier.

I did keep working on it on and I managed to invent a DocFX issue that did not need to exsist in the first place, and I spent too much time trying to fix it up until I found the actual cause and the current approach works now.

The ASF links (and also the navigation links) are now in the docfx metadata. As for the apidocs, they are in the docfx.global.subsites.json and I added it docfx.site.json's globalMetadataFiles so it appears everywhere, both the main page and the /api pages and it is updated only at one place. (Added auto-update to the docs.ps1)

If you want a different structure where these links values are, just let me know and I will adjust it quickly!
Also, I left the exsisting JS-based runtime injection, I did not notice any issue, but let me know if you want anything about that.

I just realized that the site was 'left behind' as I was troubleshooting the apidocs, and I will quickly make them more uniform by modify the site to match how the links are stored and called at the navbar partial. Then I will revert it from draft.

…the apidocs

- Replaced array-based navigation link config in docfx.json with individual metadata fields for each link.
- Removed the navbar-links.tmpl.partial template as it become no longer necessary.
- Updated navbar.tmpl.partial to render links directly from the new fields.
@zka26 zka26 marked this pull request as ready for review January 11, 2026 23:10
@zka26

zka26 commented Jan 11, 2026

Copy link
Copy Markdown
Contributor Author

Now the site and apidocs are more uniform in how the navbar links are stored and rendered.

Apologies about the delay again and thanks for your patience!

…ined titles; shorten header link text so it all fits

@paulirwin paulirwin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay on this, looks good! I pushed up some small changes to whitespace, a minor accessibility fix, adding in the a tag titles to match the other file in the site project, and shortened the header link text so that it fits without wrapping.

Before:
Image

After:

Image

@paulirwin paulirwin merged commit b6a3243 into apache:master Apr 10, 2026
2 checks passed
@paulirwin paulirwin added the notes:website-or-documentation Documentation or website changes label Apr 15, 2026
paulirwin added a commit to paulirwin/lucene.net that referenced this pull request Jun 8, 2026
* site: add links to configuration file

Introduces _navbarLinks and _asfLinks configuration in docfx.json including the correct event page link. Adds a new navbar-links partial to render these links and updates the main navbar partial to include it.

* Fix breadcrumb styles in main.css

Now the Api / Libraries works as intended, not splitting into two lines and the slash is restored.

* apidocs: add links to configuration file

Introduces _navbarLinks and _asfLinks configuration in docfx.global.json including the correct event page link. Adds a new navbar-links partial to render these links and updates the main navbar partial to include it.

* apidocs: move links to separate .json

- Added docfx.links.json to organize navigation and ASF links separately. It now includes the links.
- Modified docfx.site.json to include docfx.links.json in globalMetadataFiles.

* Update doc links to use latest Lucene.Net version

Added docfx.links.json to metadata and updated navbar links to point to the latest CI documentation. Enhanced docs.ps1 to automatically update both docfx.links.json with the current LuceneNetVersion during the build process, ensuring navigation and API links always reference the correct version.

* Fix build error by storing navbar links differently and links not appearing inside the apidocs.

- Refactor navbar links to use individual properties and moved to the subsite.json.
- docfx.links.json and navbar-links.tmpl.partial removed.
- docs.ps1 restored as it was originally.
- Removed the link json file reference from the metadata docfx.core.json.

* Update CLI doc link with auto-update and minor changes.

- Change _navCliHref to absolute URL for published CLI docs
- Removed the links jsons globalMetadataFiles leftover and the subsite one was added instead.
- Auto-update CLI doc version in docs.ps1
- Improve navbar template indentation for readability and removed a debug message from it.

* Refactor navbar config to use individual metadata to be uniform with the apidocs

- Replaced array-based navigation link config in docfx.json with individual metadata fields for each link.
- Removed the navbar-links.tmpl.partial template as it become no longer necessary.
- Updated navbar.tmpl.partial to render links directly from the new fields.

* PR feedback: whitespace fixes; a11y improvement for ASF link; use defined titles; shorten header link text so it all fits

---------

Co-authored-by: Paul Irwin <paulirwin@gmail.com>
paulirwin added a commit to paulirwin/lucene.net that referenced this pull request Jun 16, 2026
* site: add links to configuration file

Introduces _navbarLinks and _asfLinks configuration in docfx.json including the correct event page link. Adds a new navbar-links partial to render these links and updates the main navbar partial to include it.

* Fix breadcrumb styles in main.css

Now the Api / Libraries works as intended, not splitting into two lines and the slash is restored.

* apidocs: add links to configuration file

Introduces _navbarLinks and _asfLinks configuration in docfx.global.json including the correct event page link. Adds a new navbar-links partial to render these links and updates the main navbar partial to include it.

* apidocs: move links to separate .json

- Added docfx.links.json to organize navigation and ASF links separately. It now includes the links.
- Modified docfx.site.json to include docfx.links.json in globalMetadataFiles.

* Update doc links to use latest Lucene.Net version

Added docfx.links.json to metadata and updated navbar links to point to the latest CI documentation. Enhanced docs.ps1 to automatically update both docfx.links.json with the current LuceneNetVersion during the build process, ensuring navigation and API links always reference the correct version.

* Fix build error by storing navbar links differently and links not appearing inside the apidocs.

- Refactor navbar links to use individual properties and moved to the subsite.json.
- docfx.links.json and navbar-links.tmpl.partial removed.
- docs.ps1 restored as it was originally.
- Removed the link json file reference from the metadata docfx.core.json.

* Update CLI doc link with auto-update and minor changes.

- Change _navCliHref to absolute URL for published CLI docs
- Removed the links jsons globalMetadataFiles leftover and the subsite one was added instead.
- Auto-update CLI doc version in docs.ps1
- Improve navbar template indentation for readability and removed a debug message from it.

* Refactor navbar config to use individual metadata to be uniform with the apidocs

- Replaced array-based navigation link config in docfx.json with individual metadata fields for each link.
- Removed the navbar-links.tmpl.partial template as it become no longer necessary.
- Updated navbar.tmpl.partial to render links directly from the new fields.

* PR feedback: whitespace fixes; a11y improvement for ASF link; use defined titles; shorten header link text so it all fits

---------

Co-authored-by: Paul Irwin <paulirwin@gmail.com>
jpapiez pushed a commit to OlyForge3D/PrintFarmer that referenced this pull request Jun 30, 2026
Updated [Lucene.Net](https://github.com/apache/lucenenet) from
4.8.0-beta00017 to 4.8.0-beta00018.

<details>
<summary>Release notes</summary>

_Sourced from [Lucene.Net's
releases](https://github.com/apache/lucenenet/releases)._

## 4.8.0-beta00018

> This is a maintenance update that upgrades ICU4N to the latest
version, since several serious concurrency and resource loading bugs
have been patched since the last Lucene.NET release.

<!-- Release notes generated using configuration in .github/release.yml
at Lucene.Net_4_8_0_beta00018 -->

## What's Changed
### 🐞 Bug Fixes
* FuzzyQuery produces a wrong result when prefix is equal to the term
length by @​paulirwin in apache/lucenenet#1002
* Validate PatternParser DTDs against expected name by @​paulirwin in
apache/lucenenet#1358
* Validate file paths for FSDirectory and Replicator by @​paulirwin in
apache/lucenenet#1357
* Bumped ICU4N to 60.1.0-alpha.440 by @​NightOwl888 in
apache/lucenenet#1353
* ShingleFilter produces invalid queries by @​tohidemyname in
apache/lucenenet#946
* Fix SegmentInfos replace doesn't update userData by @​tohidemyname in
apache/lucenenet#948
### 🚀 Performance Improvements
* SWEEP: Replace J2N's TripleShift call with C# 11's unsigned right
shift operator by @​paulirwin in
apache/lucenenet#1007
### 🏆 Improvements
* Added "Improvements" Category for Release Notes by @​NightOwl888 in
apache/lucenenet#1015
### 📄 Website and API Documentation
* website/site/.htaccess - bug fix by removing BOM and update to
beta0017 redirection by @​rclabo in
apache/lucenenet#1005
* Updated .htaccess copy and release procedure by @​NightOwl888 in
apache/lucenenet#1010
* Added GitHub Automation for Release Notes by @​NightOwl888 in
apache/lucenenet#1011
* fix: Render ASF policy links in static HTML footer by @​rbowen in
apache/lucenenet#1303
* Fix/apidocs breadcrumb toc asf by @​zka26 in
apache/lucenenet#1232
* README: fix typo MacOS -> macOS by @​jbampton in
apache/lucenenet#1179
* Added ASF-required links using drop-down menu and unified navigation
by @​zka26 in apache/lucenenet#1198
* fix: Self-host all external website dependencies by @​mmafrar in
apache/lucenenet#1197
* Fix typos by @​jbampton in
apache/lucenenet#1177
* Replace lucene.testSettings.config references with
lucene.testsettings.json by @​paulirwin in
apache/lucenenet#1035

## New Contributors
* @​jbampton made their first contribution in
apache/lucenenet#1177
* @​mmafrar made their first contribution in
apache/lucenenet#1197
* @​rbowen made their first contribution in
apache/lucenenet#1303
* @​tohidemyname made their first contribution in
apache/lucenenet#946
* @​zka26 made their first contribution in
apache/lucenenet#1198

**Full Changelog**:
apache/lucenenet@Lucene.Net_4_8_0_beta00017...Lucene.Net_4_8_0_beta00018

Commits viewable in [compare
view](apache/lucenenet@Lucene.Net_4_8_0_beta00017...Lucene.Net_4_8_0_beta00018).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Lucene.Net&package-manager=nuget&previous-version=4.8.0-beta00017&new-version=4.8.0-beta00018)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jpapiez pushed a commit to OlyForge3D/PrintFarmer that referenced this pull request Jun 30, 2026
…596)

Updated [Lucene.Net.QueryParser](https://github.com/apache/lucenenet)
from 4.8.0-beta00017 to 4.8.0-beta00018.

<details>
<summary>Release notes</summary>

_Sourced from [Lucene.Net.QueryParser's
releases](https://github.com/apache/lucenenet/releases)._

## 4.8.0-beta00018

> This is a maintenance update that upgrades ICU4N to the latest
version, since several serious concurrency and resource loading bugs
have been patched since the last Lucene.NET release.

<!-- Release notes generated using configuration in .github/release.yml
at Lucene.Net_4_8_0_beta00018 -->

## What's Changed
### 🐞 Bug Fixes
* FuzzyQuery produces a wrong result when prefix is equal to the term
length by @​paulirwin in apache/lucenenet#1002
* Validate PatternParser DTDs against expected name by @​paulirwin in
apache/lucenenet#1358
* Validate file paths for FSDirectory and Replicator by @​paulirwin in
apache/lucenenet#1357
* Bumped ICU4N to 60.1.0-alpha.440 by @​NightOwl888 in
apache/lucenenet#1353
* ShingleFilter produces invalid queries by @​tohidemyname in
apache/lucenenet#946
* Fix SegmentInfos replace doesn't update userData by @​tohidemyname in
apache/lucenenet#948
### 🚀 Performance Improvements
* SWEEP: Replace J2N's TripleShift call with C# 11's unsigned right
shift operator by @​paulirwin in
apache/lucenenet#1007
### 🏆 Improvements
* Added "Improvements" Category for Release Notes by @​NightOwl888 in
apache/lucenenet#1015
### 📄 Website and API Documentation
* website/site/.htaccess - bug fix by removing BOM and update to
beta0017 redirection by @​rclabo in
apache/lucenenet#1005
* Updated .htaccess copy and release procedure by @​NightOwl888 in
apache/lucenenet#1010
* Added GitHub Automation for Release Notes by @​NightOwl888 in
apache/lucenenet#1011
* fix: Render ASF policy links in static HTML footer by @​rbowen in
apache/lucenenet#1303
* Fix/apidocs breadcrumb toc asf by @​zka26 in
apache/lucenenet#1232
* README: fix typo MacOS -> macOS by @​jbampton in
apache/lucenenet#1179
* Added ASF-required links using drop-down menu and unified navigation
by @​zka26 in apache/lucenenet#1198
* fix: Self-host all external website dependencies by @​mmafrar in
apache/lucenenet#1197
* Fix typos by @​jbampton in
apache/lucenenet#1177
* Replace lucene.testSettings.config references with
lucene.testsettings.json by @​paulirwin in
apache/lucenenet#1035

## New Contributors
* @​jbampton made their first contribution in
apache/lucenenet#1177
* @​mmafrar made their first contribution in
apache/lucenenet#1197
* @​rbowen made their first contribution in
apache/lucenenet#1303
* @​tohidemyname made their first contribution in
apache/lucenenet#946
* @​zka26 made their first contribution in
apache/lucenenet#1198

**Full Changelog**:
apache/lucenenet@Lucene.Net_4_8_0_beta00017...Lucene.Net_4_8_0_beta00018

Commits viewable in [compare
view](apache/lucenenet@Lucene.Net_4_8_0_beta00017...Lucene.Net_4_8_0_beta00018).
</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

notes:website-or-documentation Documentation or website changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Website caching issue

2 participants